-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add TNT for CTF #1284
base: master
Are you sure you want to change the base?
Add TNT for CTF #1284
Conversation
This PR is ready to review. Though I cannot tell how to fix the remaining luacheck lints. There is one issue though: When TNT explodes and removes blocks, it gives score to the owner according to the number of removed blocks. Like 70 removed blocks give 70 score. And since a TNT can only be placed near enemy base, this is the score for enemy base sabotage and/or destruction. However, 70 score seems too much to give away. Perhaps I divide the removed blocks number by something like... 5? |
Does it damage players? IMO it should be used to remove blocks, not kill people. That's what nades are for |
Also, IDK about awarding score... |
I'd rather not award score, and I'd like to keep the rules for placement as simple and non-restrictive as possible. It would probably be best to restrict this change to nade fight at first |
I'm in favor of awarding score as TNT can be placed only near an enemy base. And it is most likely enemy base destruction which is score-worthy and contributes to the goal of game which is CTF. Edit: But if you think 1 score per destroyed block is too much, we can reduce it to like 1/5 per block or even 1/10 |
BTW, what do you think about adding a new stat: "Enemy base destruction". Could be number of blocks removed from the enemy base. |
What count's as an "enemy base" tho, unfourtunatly, i can see how this could be exploited |
Hello. It cannot be exploited since TNTs can be placed only close to enemy base. |
This PR aims to add TNTs to CTF, one of popular suggestions made by players.